-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add DevinfoInit to Relevant CC13xx apps #22184
Conversation
PR #22184: Size comparison from 7870328 to e76f8b4 Increases (1 build for telink)
Full report (11 builds for bl602, k32w, linux, mbed, nrfconnect, telink)
|
PR #22184: Size comparison from 48a4797 to a800167 Increases above 0.2%:
Increases (6 builds for cc13x2_26x2, nrfconnect, psoc6)
Decreases (8 builds for cc13x2_26x2, cyw30739, psoc6, telink)
Full report (43 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, telink)
|
a800167
to
7ba285f
Compare
616e0b7
to
c525f22
Compare
PR #22184: Size comparison from 3d7cc78 to c525f22 Increases above 0.2%:
Increases (11 builds for cc13x2_26x2, cyw30739, efr32, esp32, nrfconnect, psoc6)
Decreases (7 builds for bl602, cc13x2_26x2, psoc6, telink)
Full report (43 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, telink)
|
@adabreuti - it looks like this needs restyle fixes. |
c525f22
to
8c6e222
Compare
Resolved |
PR #22184: Size comparison from 934adfd to 2b4fa00 Increases above 0.2%:
Increases (11 builds for bl602, cc13x2_26x2, cyw30739, nrfconnect, psoc6, telink)
Decreases (6 builds for cc13x2_26x2, efr32, psoc6)
Full report (43 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, telink)
|
@andy31415 It appears as if all PR's are currently not able to be merged due to the "Test Suites - Darwin" case failing. Is it possible for this to be overridden as this PR is platform specific? |
Retrying runs. |
1.0 accepted as platform-specific change. |
* Add DevinfoInit to CC13xx all clusters * Add Devinfo init to lock-app * Update style
Problem
Device info cluster is not initialized at the correct location.
Change overview
Initialize dev info cluster
Testing
Verified labels are able to be correctly created in all clusters cc13xx implementation and that the projects build.